Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing intermittent OSX test failure issue due to low default filehandle limit #613

Merged
merged 4 commits into from
Dec 1, 2022

Conversation

brucek
Copy link
Contributor

@brucek brucek commented Dec 1, 2022

Motivation

On OSX, tests sometimes fail with an error like "panicked at 'Could not create ledger_db: LMDB: Too many open files'"

In this PR

Test Plan

  • follow README instructions
  • run as usual via ./tools/test.sh

Future Work

@brucek brucek requested review from briancorbin and cxloe December 1, 2022 01:53
@codecov-commenter
Copy link

Codecov Report

Base: 59.28% // Head: 59.29% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (2463abd) compared to base (0597e86).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #613   +/-   ##
========================================
  Coverage    59.28%   59.29%           
========================================
  Files           82       82           
  Lines        11503    11503           
  Branches      1858     1858           
========================================
+ Hits          6820     6821    +1     
  Misses        3149     3149           
+ Partials      1534     1533    -1     
Impacted Files Coverage Δ
full-service/src/db/wallet_db.rs 24.24% <0.00%> (+0.75%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

README.md Show resolved Hide resolved
@cxloe cxloe self-requested a review December 1, 2022 02:06
brucek and others added 2 commits November 30, 2022 18:08
Co-authored-by: cxloe <77425619+cxloe@users.noreply.github.com>
@brucek brucek closed this Dec 1, 2022
@brucek brucek reopened this Dec 1, 2022
@brucek brucek merged commit ad940ae into develop Dec 1, 2022
@brucek brucek deleted the brucek/#183929476/fix-test-failures-osx branch December 1, 2022 02:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants